Skip to content

Disable two of unstable FTS case to make CI happy#532

Closed
jiaqizho wants to merge 1 commit intoapache:mainfrom
jiaqizho:disable-two-fts-unstable-cases
Closed

Disable two of unstable FTS case to make CI happy#532
jiaqizho wants to merge 1 commit intoapache:mainfrom
jiaqizho:disable-two-fts-unstable-cases

Conversation

@jiaqizho
Copy link
Copy Markdown
Contributor

fix #ISSUE_Number


Change logs

In different linux distributions, these two tests often fail due to time differences.

In this change, i disable two of unstable FTS case to make CI happy.

We should keep the test files until CBDB merge all GPDB commits.

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Contributor's Checklist

Here are some reminders and checklists before/when submitting your pull request, please check them:

  • Make sure your Pull Request has a clear title and commit message. You can take git-commit template as a reference.
  • Sign the Contributor License Agreement as prompted for your first-time contribution(One-time setup).
  • Learn the coding contribution guide, including our code conventions, workflow and more.
  • List your communication in the GitHub Issues or Discussions (if has or needed).
  • Document changes.
  • Add tests for the change
  • Pass make installcheck
  • Pass make -C src/test installcheck-cbdb-parallel
  • Feel free to request cloudberrydb/dev team for review and approval when your PR is ready🥳

@jiaqizho jiaqizho requested a review from my-ship-it July 24, 2024 11:26
@my-ship-it
Copy link
Copy Markdown
Contributor

Already merged due to github bug. So close this PR.

@my-ship-it my-ship-it closed this Jul 24, 2024
oppenheimer01 pushed a commit to oppenheimer01/cloudberrydb that referenced this pull request Jul 26, 2024
In different linux distributions, these two tests often fail due to time differences.
In this change, disable two of unstable FTS case to make CI happy.
We should keep the test files until CBDB merge all GPDB commits.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants